Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

samples: radio_test: Fix lookup table with time per byte for 4Mbit mode #18868

Merged
merged 1 commit into from
Nov 14, 2024
Merged

samples: radio_test: Fix lookup table with time per byte for 4Mbit mode #18868

merged 1 commit into from
Nov 14, 2024

Conversation

maje-emb
Copy link
Contributor

Fixed the lookup table to correctly handle byte durations in 4Mbit mode.

@maje-emb
samples: radio_test: Fix lookup table with time per byte for 4Mbit mode
1075c0c 
Fixed the lookup table to correctly handle byte durations in 4Mbit mode.

Ref: NCSDK-30284

Signed-off-by: Marcin Jelinski <[email protected]>
@maje-emb maje-emb requested a review from a team as a code owner November 13, 2024 15:55
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 13, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 13, 2024 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 1

Inputs:

Sources:

sdk-nrf: PR head: 1075c0c48f60f97c9ee726544a35131de6d64406

more details

sdk-nrf:

PR head: 1075c0c48f60f97c9ee726544a35131de6d64406
merge base: 83118d7d268baf9fb13c5260023fcd9b6e3aa4b6
target head (main): 6985c36f47e84320180dbf5921f2c8be25f1ee37
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (1) 
samples
│  ├── peripheral
│  │  ├── radio_test
│  │  │  ├── src
│  │  │  │  │ radio_test.c

Outputs:

Toolchain

Version: f51bdba1d9
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:f51bdba1d9_912848a074

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 53
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-ble_samples
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi

Note: This message is automatically posted and updated by the CI

@grochu grochu added this to the 2.8.0 milestone Nov 14, 2024
grochu
grochu approved these changes Nov 14, 2024
View reviewed changes
Copy link
Contributor

@grochu grochu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to have a fix before tagging

@maje-emb maje-emb added the backport v2.8-branch auto-create a PR with same commits to v2.8-branch label Nov 14, 2024
@nordicjm nordicjm merged commit d839e3c into nrfconnect:main Nov 14, 2024
15 checks passed
@NordicBuilder NordicBuilder mentioned this pull request Nov 14, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@doki-nordic doki-nordic doki-nordic approved these changes

@grochu grochu grochu approved these changes

@rakons rakons Awaiting requested review from rakons

Assignees
No one assigned
Labels
backport v2.8-branch auto-create a PR with same commits to v2.8-branch changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.
Projects
None yet
Milestone
2.8.0
Development

Successfully merging this pull request may close these issues.
5 participants
@maje-emb @NordicBuilder @grochu @doki-nordic @nordicjm